Skip to content

Response status updates#531

Draft
MichaelGHSeg wants to merge 22 commits intomasterfrom
response-status-updates
Draft

Response status updates#531
MichaelGHSeg wants to merge 22 commits intomasterfrom
response-status-updates

Conversation

@MichaelGHSeg
Copy link
Contributor

No description provided.

MichaelGHSeg and others added 22 commits December 3, 2025 15:03
* Changing release plugin

* Deploy working

* Rolling back to snapshot

* [maven-release-plugin] prepare release analytics-parent-3.5.4

* [maven-release-plugin] prepare for next development iteration

* putting back release instructions
- Add Authorization header with Basic auth to all requests
- Reduce retry backoff times for faster recovery (100ms base, 1min cap)
- Respect Retry-After header with 5-minute max cap
- Always include X-Retry-Count header in requests
- Consolidate SegmentService upload methods
- Increase max flush attempts to accommodate shorter backoff times

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove 413 (Payload Too Large) from retryable status codes
  Payload size won't shrink on retry, so retrying is pointless

- Verify 503 Retry-After handling matches spec
  503 checks Retry-After header first, falls back to exponential backoff

- Add test for 413 non-retryable behavior
- Add test for Retry-After delay mechanism (validates cap logic)
  The 300-second cap is enforced at AnalyticsClient.java:556-558

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant